[feat](function) Add limit parameter support for SPLIT_BY_STRING#60892
[feat](function) Add limit parameter support for SPLIT_BY_STRING#60892floation-cutie wants to merge 11 commits intoapache:masterfrom
Conversation
|
Thank you for your contribution to Apache Doris. Please clearly describe your PR:
|
|
run buildall |
TPC-H: Total hot run time: 28972 ms |
TPC-DS: Total hot run time: 183866 ms |
125e522 to
e89a95b
Compare
|
run buildall |
TPC-H: Total hot run time: 28664 ms |
TPC-DS: Total hot run time: 182736 ms |
FE UT Coverage ReportIncrement line coverage |
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
25f35e6 to
5d70879
Compare
|
run buildall |
TPC-H: Total hot run time: 28721 ms |
5d70879 to
1c9eaa9
Compare
TPC-DS: Total hot run time: 182960 ms |
1c9eaa9 to
e9c53c3
Compare
|
run buildall |
TPC-H: Total hot run time: 28535 ms |
TPC-DS: Total hot run time: 182599 ms |
FE UT Coverage ReportIncrement line coverage |
33b32c5 to
ed69d1c
Compare
6e417c8 to
4d33233
Compare
|
run buildall |
FE UT Coverage ReportIncrement line coverage |
TPC-H: Total hot run time: 26969 ms |
TPC-DS: Total hot run time: 169776 ms |
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
|
run p0 |
|
run cloud_p0 |
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
… coercion for limit parameter
e47aea1 to
1584a61
Compare
|
run buildall |
TPC-H: Total hot run time: 26795 ms |
TPC-DS: Total hot run time: 169112 ms |
FE UT Coverage ReportIncrement line coverage |
BE UT Coverage ReportIncrement line coverage Increment coverage report
|
BE Regression && UT Coverage ReportIncrement line coverage Increment coverage report
|
What problem does this PR solve?
Issue Number: close #55788
Related PR: #55788
Problem Summary:
This PR implements the enhancement requested in issue #55788, adding an optional third parameter
limitto theSPLIT_BY_STRINGfunction. Thelimitparameter controls the maximum number of splits to perform, limiting the size of the returned array.Solution
limittoSPLIT_BY_STRING(<str>, <separator>[, <limit>])limitis positive: split from left, at mostlimitsplits, returning at mostlimit + 1elementslimitis non-positive or not specified: split all occurrences (backward compatible)function_string.cppandfunction_string.hSplitByString.javaandStringArithmetic.javaExamples:
Release note
[Enhancement] Add optional
limitparameter toSPLIT_BY_STRINGfunction to control maximum number of splits.Check List (For Author)
Test
Behavior changed:
Does this need documentation?
Check List (For Reviewer who merge this PR)